Skip to content

MINOR: Remove duplicate test dependency declarations for clients module#12764

Merged
C0urante merged 1 commit into
apache:trunkfrom
C0urante:clients-test-deps
Oct 21, 2022
Merged

MINOR: Remove duplicate test dependency declarations for clients module#12764
C0urante merged 1 commit into
apache:trunkfrom
C0urante:clients-test-deps

Conversation

@C0urante

Copy link
Copy Markdown
Contributor

#10528 accidentally added a duplicate dependency on Log4j; this change removes it, and also groups the dependencies by scope to tidy things up a bit.

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

@mimaison mimaison left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ijuma ijuma left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGRM

@C0urante
C0urante merged commit 215d4f9 into apache:trunk Oct 21, 2022
@C0urante
C0urante deleted the clients-test-deps branch October 21, 2022 16:18
guozhangwang pushed a commit to guozhangwang/kafka that referenced this pull request Jan 25, 2023
…le (apache#12764)

Reviewers: Mickael Maison <mickael.maison@gmail.com>, Ismael Juma <ismael@juma.me.uk>
rutvijmehta-harness pushed a commit to rutvijmehta-harness/kafka that referenced this pull request Feb 9, 2024
…le (apache#12764)

Reviewers: Mickael Maison <mickael.maison@gmail.com>, Ismael Juma <ismael@juma.me.uk>
rutvijmehta-harness added a commit to rutvijmehta-harness/kafka that referenced this pull request Feb 9, 2024
…le (apache#12764) (#79)

Reviewers: Mickael Maison <mickael.maison@gmail.com>, Ismael Juma <ismael@juma.me.uk>

Co-authored-by: Chris Egerton <chrise@aiven.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants